Skip to content

Restore social sharing modal#106

Merged
Rayan9064 merged 2 commits into
mainfrom
restore-social-sharing-modal
Jul 12, 2026
Merged

Restore social sharing modal#106
Rayan9064 merged 2 commits into
mainfrom
restore-social-sharing-modal

Conversation

@Rayan9064

@Rayan9064 Rayan9064 commented Jul 12, 2026 •
edited by coderabbitai Bot
Loading

Copy link
Copy Markdown
Owner

Summary by CodeRabbit

  • New Features
    • Added a sharing step after creating a resource.
    • Share resources on X, LinkedIn, and Facebook.
    • Copy resource links to the clipboard with confirmation feedback.
    • Close the sharing dialog or finish sharing to complete the creation flow.

@vercel

vercel Bot commented Jul 12, 2026

Copy link
Copy Markdown
Contributor

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
dumpit Ready Ready Preview, Comment Jul 12, 2026 6:40am

@Rayan9064 Rayan9064 merged commit e7e9f26 into main Jul 12, 2026
4 checks passed
@Rayan9064 Rayan9064 deleted the restore-social-sharing-modal branch July 12, 2026 06:40
@coderabbitai

coderabbitai Bot commented Jul 12, 2026 •
edited
Loading

Copy link
Copy Markdown

Review Change Stack

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: e842de2a-bdb0-4749-a7c8-603e27f16045

📥 Commits

Reviewing files that changed from the base of the PR and between e08a0b1 and 89b38d5.

📒 Files selected for processing (2)
  • app/components/AddResource.tsx
  • app/components/ui/ShareModal.tsx
📝 Walkthrough

Walkthrough

Changes

Resource sharing flow

Layer / File(s) Summary
ShareModal actions and presentation
app/components/ui/ShareModal.tsx		 Adds a conditional sharing dialog with X/Twitter, LinkedIn, Facebook, public-link copying, transient copied-state feedback, and close controls.
Post-submit modal integration
app/components/AddResource.tsx		 Stores submitted resource details, opens the ShareModal after creation, and calls onSuccess after the modal closes.

Estimated code review effort: 3 (Moderate) | ~20 minutes

Sequence Diagram(s)

sequenceDiagram
  participant AddResource
  participant ShareModal
  participant SocialPlatforms
  participant Clipboard
  AddResource->>ShareModal: Open with resource title, note, and link
  ShareModal->>SocialPlatforms: Open sharing URL
  ShareModal->>Clipboard: Copy public link
  ShareModal->>AddResource: Close modal
  AddResource->>AddResource: Invoke onSuccess
Loading

Possibly related PRs

✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch restore-social-sharing-modal

Warning

There were issues while running some tools. Please review the errors and either fix the tool's configuration or disable the tool if it's a critical failure.

🔧 ESLint

If the error stems from missing dependencies, add them to the package.json file. For unrecoverable errors (e.g., due to private dependencies), disable the tool in the CodeRabbit configuration.

app/components/AddResource.tsx

ESLint skipped: missing config or dependency (missing-dependency). The ESLint configuration references a package that is not available in the sandbox.

app/components/ui/ShareModal.tsx

ESLint skipped: the ESLint configuration for this file references a package that is not available in the sandbox.

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@Rayan9064 Rayan9064 mentioned this pull request Jul 12, 2026
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@Rayan9064